Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

#876 Major contribution to gnucash-android #882

Open
wants to merge 77 commits into
base: develop
Choose a base branch
from

Conversation

JeanGarf
Copy link

@JeanGarf JeanGarf commented Jun 19, 2020

Hi,

This is one of my main contribution to GnucashAndroid.

It corrects #876, but also lots of other errors like

You can see all my contributions together on the following branch :
https://github.com/JeanGarf/gnucash-android/tree/tw_develop

If you want to get the corresponding apk, you can get it on my web site :
http://thierryw777.free.fr/index.php/bureautique/comptabilite/gnucash-android/

See you,

…dows. isChecked() : true == CREDIT. CREDIT => red, DEBIT => green
…nsure storing of absolute values in splits
@JeanGarf JeanGarf changed the title #876 balance liability #876 Major contribution to gnucash-android Jun 19, 2020
@strixaluco
Copy link

Thank you for your contribution, @JeanGarf. I immediately thought about writing to @codinguser about your PR, have you tried getting in touch with him, possibly by email or other means? We can't let this solid amount of work be stuck in limbo.

@JeanGarf
Copy link
Author

JeanGarf commented Aug 6, 2020

Hi strixaluco,

Yes I tried to send email to @codinguser, but I didn't get any response.

I'm new to contribute to open source project and to Android and therefore I don't know what to do.

I can perhaps upload the apk corresponding to my cumulative tw_develop branch, but I don't if it is a good idea or not ?

@strixaluco
Copy link

It's a pity there was no response. I've also replied on his tweet about GnuCash from about 3 years ago, mentioning stalled PRs, but who knows if that will have any effect.
I'm sure that GitHub lurkers like us will appreciate your own build, but that doesn't seem like sustainable solution given the fact, that there's no way to submit updates to Google Play.
Maybe people from the main GnuCash team can give us a hint on the best way forward.

@strixaluco
Copy link

Seems that another contributor managed to reach @codinguser and he promised to look into merging all the PRs.

@JeanGarf
Copy link
Author

Hi @strixaluco

I added a page on my web site to allow downloading of the apk with all my contributions
http://thierryw777.free.fr/index.php/bureautique/comptabilite/gnucash-android/

If you're interested to test or use it, tell me if it works fine.

See you

@strixaluco
Copy link

Really appreciate your efforts, @JeanGarf. Will try your build and will report if anything.

@Thumas
Copy link

Thumas commented Aug 23, 2020

The enhancements are pretty useful!
But you didn't adjust the UI tests to your new searchable spinner. Running a slightly adjusted multi-currency test reveals that your spinner seems to trigger the onItemSelectedListener even if you only search for an account without actively selecting it.

  1. Add three accounts with different currencies to an empty database.
  2. Select one of them, create a new transaction.
  3. Enter a value
  4. Click on the transaction account, then your searchable spinner appears.
  5. Enter something that does not trigger the automatic choice and then delete it completely.
  6. The transfer funds dialog appears, although you didn't select an account yet.

@JeanGarf
Copy link
Author

The enhancements are pretty useful!
But you didn't adjust the UI tests to your new searchable spinner. Running a slightly adjusted multi-currency test reveals that your spinner seems to trigger the onItemSelectedListener even if you only search for an account without actively selecting it.

1. Add three accounts with different currencies to an empty database.

2. Select one of them, create a new transaction.

3. Enter a value

4. Click on the transaction account, then your searchable spinner appears.

5. Enter something that does not trigger the automatic choice and then delete it completely.

6. The transfer funds dialog appears, although you didn't select an account yet.

I Thumas,

I'm sorry, I'm very new to android dev and I didn't success to run UI tests.
I even had to remove lines 173 to 178 (spoon) in app/build.gradle in order to compile !
I don't know why.

Could you explain me how to make them work ?

Thanks

@aramaicus
Copy link

Hi @strixaluco

I added a page on my web site to allow downloading of the apk with all my contributions
http://thierryw777.free.fr/index.php/bureautique/comptabilite/gnucash-android/

If you're interested to test or use it, tell me if it works fine.

See you

Could you please re-upload the apk? The link from Google Drive isn't working.

@JeanGarf
Copy link
Author

Hi @strixaluco
I added a page on my web site to allow downloading of the apk with all my contributions
http://thierryw777.free.fr/index.php/bureautique/comptabilite/gnucash-android/
If you're interested to test or use it, tell me if it works fine.
See you

Could you please re-upload the apk? The link from Google Drive isn't working.

Hi, I'm sorry, I made a small change in the app and forget to update my link.

Could you try again and tell me if it works fine ?

See you

P.S. : The change is in the Account chooser is the following : When you type characters in the search field, account list is filtered. If only one account remains in the filtered list, it is not automatically selected anymore. You have to click on it to choose it.
This allows you to check that this really the account you want to select.

@albert2004
Copy link

Version available in provided link have one issue to me.
When I use my currency (PLN) and provide amount in transaction, for example 130,67PLN it works fine.
But when I enter transaction again for edit, amount is rounded (131PLN). This was not happening with official version.
I write it here, since I don't see "issues" section in your fork - maybe disabled on purpose ;)

@aramaicus
Copy link

aramaicus commented May 30, 2021 via email

@JeanGarf
Copy link
Author

JeanGarf commented Jun 5, 2021

Version available in provided link have one issue to me.
When I use my currency (PLN) and provide amount in transaction, for example 130,67PLN it works fine.
But when I enter transaction again for edit, amount is rounded (131PLN). This was not happening with official version.
I write it here, since I don't see "issues" section in your fork - maybe disabled on purpose ;)

Hi, thank for reporting this issue.
I also have the same problem. I'll correct it in the following weeks.

See you

…not currency symbol because it shall be evaluated (in order to allows arithmetic expressions as value)
@JeanGarf
Copy link
Author

Version available in provided link have one issue to me.
When I use my currency (PLN) and provide amount in transaction, for example 130,67PLN it works fine.
But when I enter transaction again for edit, amount is rounded (131PLN). This was not happening with official version.
I write it here, since I don't see "issues" section in your fork - maybe disabled on purpose ;)

Hi albert2004 and aramaicus,

I've corrected my mistake. You can get the corrected apk on my web site here :

http://thierryw777.free.fr/index.php/bureautique/comptabilite/gnucash-android/

Thanks for having reported the issue.

Bye

@aramaicus
Copy link

aramaicus commented Jun 10, 2021 via email

@albert2004
Copy link

Hi JeanGarf,
I built from sources with fix.
Also I corrected one language typo.
For Android banking I do not except anything more. Finally it works like it should.
Good luck with push!

@aramaicus
Copy link

aramaicus commented Jun 15, 2021 via email

@aramaicus
Copy link

I have and file created on Gnucash on Android 2 years ago. That file works with Gnucash desktop but doesn't work with Gnucash for Android.

When I try to open it, I receive that message:

"An error occurred while importing the GnuCash accounts Unable to parse transaction time - 2019-03-31 15:00:13"

I'm using GnucashAndroid_v2.4.1-dev4_535f3d53.apk

I don't mind sharing that file. There is no confidential information in it.

If you care about it, make me know so I'll send you the file.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants